Looks great! are you able to avoid the whitespace changes here at all? Would be nice to include just the changes you made to the code.

💃

black did that I think. There's always git diff -b ;)

Can you ensure you're using the same version of black as is in the repo?

ah I guess we don't pin it. I have 19.10b0, how about you?

black, version 20.8b1
In general, why don't we pin stuff always? It seems to break stuff often when pip repos are upgraded.

All it "requires" (ROFL) is a little pip freeze > requirements.txt

well, it's helpful to get a warning when a newer version of pandas is incompatible with our stuff... but yeah let's pin black to 19.10b0 for now. Can you do that in this PR and rerun it on your changes?

Ah, I see it's nice that it shows that automatically. What if releases are pinned but master isn't? Then the warnings happen in master but the releases always work.
Anyhow I can try pinning black 👍

Unfortunately running black version 19.10b0 doesn't change the file packages/python/plotly/_plotly_utils/basevalidators.py, but I pinned it in optional-requirements.txt.

OK, can you please undo the whitespace changes and merge then?